object_store: Add enabled-by-default "fs" feature#6636
object_store: Add enabled-by-default "fs" feature#6636crepererum merged 1 commit intoapache:mainfrom
Conversation
|
/cc @tustvold |
tustvold
added
api-change
|
I had a quick look and this looks plausible, however, it is a breaking change and will therefore need to wait for the next major release |
Turbo87
force-pushed
the
object_store/optional-fs
branch
from
799e4cf to
77c3931
Compare
Turbo87
force-pushed
the
object_store/optional-fs
branch
from
77c3931 to
5efbd80
Compare
Turbo87
force-pushed
the
object_store/optional-fs
branch
from
5efbd80 to
0e496ac
Compare
object_store/src/lib.rs
Outdated
There was a problem hiding this comment.
"local filesystem" should be removed from the list in the previous paragraph (GitHub doesn't let me add a comment there, but this one here:
arrow-rs/object_store/src/lib.rs
Line 69 in 13f3f21
There was a problem hiding this comment.
good catch. fixed! :)
Turbo87
force-pushed
the
object_store/optional-fs
branch
from
0e496ac to
a48eb8b
Compare
crepererum
left a comment
crepererum
left a comment
There was a problem hiding this comment.
Looks good. Please delay the merge until 0.12 though (see #6596).
I hope to be able to merge early next week |
|
0.11.2 is out, we're now on the way to 0.12 🚀 |
4 participants
Which issue does this PR close?
Closes apache/arrow-rs-object-store#62.
Rationale for this change
see apache/arrow-rs-object-store#62 😉
What changes are included in this PR?
This PR introduces a new
fsfeature for theobject_storecrate, which is enabled by default. If disabled, thelocalmodule is excluded from the crate and thewalkdirdependency is skipped.Are there any user-facing changes?
Only for users of
default-features = false, but since this crate did not have any default features before there probably won't be manyobject_storeusers that specifydefault-features = false.